From: Keir Fraser Date: Tue, 4 Dec 2007 11:52:10 +0000 (+0000) Subject: Fix xenctl_cpumap_to_cpumask. X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~14680 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/success/%22http:/www.example.com/cgi/success?a=commitdiff_plain;h=7dba6db6e9ff4c659d0e1d7719668514ecce517f;p=xen.git Fix xenctl_cpumap_to_cpumask. Signed-off-by: Keir Fraser --- diff --git a/xen/common/domctl.c b/xen/common/domctl.c index 8d26075ee6..98fa23915d 100644 --- a/xen/common/domctl.c +++ b/xen/common/domctl.c @@ -43,7 +43,8 @@ void cpumask_to_xenctl_cpumap( bitmap_long_to_byte(bytemap, cpus_addr(*cpumask), NR_CPUS); - copy_to_guest(xenctl_cpumap->bitmap, bytemap, copy_bytes); + if ( copy_bytes != 0 ) + copy_to_guest(xenctl_cpumap->bitmap, bytemap, copy_bytes); for ( i = copy_bytes; i < guest_bytes; i++ ) copy_to_guest_offset(xenctl_cpumap->bitmap, i, &zero, 1); @@ -55,15 +56,20 @@ void xenctl_cpumap_to_cpumask( unsigned int guest_bytes, copy_bytes; uint8_t bytemap[(NR_CPUS + 7) / 8]; + if ( guest_handle_is_null(xenctl_cpumap->bitmap) ) + return; + guest_bytes = (xenctl_cpumap->nr_cpus + 7) / 8; copy_bytes = min_t(unsigned int, guest_bytes, sizeof(bytemap)); - cpus_clear(*cpumask); - - if ( guest_handle_is_null(xenctl_cpumap->bitmap) ) - return; + memset(bytemap, 0, sizeof(bytemap)); - copy_from_guest(bytemap, xenctl_cpumap->bitmap, copy_bytes); + if ( copy_bytes != 0 ) + { + copy_from_guest(bytemap, xenctl_cpumap->bitmap, copy_bytes); + if ( (xenctl_cpumap->nr_cpus & 7) && (guest_bytes <= sizeof(bytemap)) ) + bytemap[guest_bytes-1] &= ~(0xff << (xenctl_cpumap->nr_cpus & 7)); + } bitmap_byte_to_long(cpus_addr(*cpumask), bytemap, NR_CPUS); }